Skip to content

(3/n) Specialize state actor bootstrap for each alloc #459

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

highker
Copy link

@highker highker commented Jul 8, 2025

Summary:
Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 8, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

@highker highker force-pushed the export-D77914042 branch from c010d74 to daaee6d Compare July 9, 2025 06:43
highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch 2 times, most recently from 78b0474 to bc43c95 Compare July 9, 2025 18:32
highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from bc43c95 to 744ba5d Compare July 9, 2025 18:37
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 9, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from 744ba5d to bb10c1b Compare July 9, 2025 18:46
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from bb10c1b to df14fc1 Compare July 10, 2025 18:44
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from df14fc1 to de13844 Compare July 10, 2025 18:45
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from de13844 to be7599e Compare July 10, 2025 18:49
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from be7599e to 96b1487 Compare July 10, 2025 19:01
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from 96b1487 to 6b2d2c6 Compare July 10, 2025 20:15
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

@highker highker force-pushed the export-D77914042 branch from 6b2d2c6 to 3f23f35 Compare July 10, 2025 21:37
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from 3f23f35 to 5baf65b Compare July 10, 2025 21:38
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

highker pushed a commit to highker/monarch that referenced this pull request Jul 10, 2025
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@highker highker force-pushed the export-D77914042 branch from 5baf65b to 81fc5c7 Compare July 10, 2025 21:43
Summary:
Pull Request resolved: pytorch-labs#459

Different alloc should have different setup of state actor.
The remote allocator should bootstrap the state actor inside the initializer.
This will be done in the follow-up diffs.

Reviewed By: kaiyuan-li

Differential Revision: D77914042
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D77914042

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 50a7cf8.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants